Skip to content

Remove examples - #49357

Merged
SunMarc merged 8 commits into
mainfrom
remove-accelerate-examples
Oct 13, 2026
Merged

Remove examples#49357
SunMarc merged 8 commits into
mainfrom
remove-accelerate-examples

Conversation

@SunMarc

@SunMarc SunMarc commented Oct 6, 2026 •
edited by github-actions Bot
Loading

Copy link
Copy Markdown
Member

🤗 Transformers CI

What does this PR do?

This PR deletes files in examples folder that we not maintained anymore and doesn't really makes sense to keep anymore:

  • pytorch folder examples (very old example files) -> need to update all the docs, CI and tests (most of the PR)
  • 3D parallel example-> do we use those no to stress test our 3D integration ? Happy to keep them otherwise cc @ArthurZucker

Note:
Don't do a deep review, i'm still checking as well but happy to have some feedback.

Btw i don't mind keeping the examples files but since we don't maintain them anymore, we still need to update the docs, tests and so on.

hf-security-analysis Bot added a commit that referenced this pull request Oct 6, 2026
@hf-security-analysis
fix(ci): harden workflow files flagged on #49357
f73874f
@hf-security-analysis hf-security-analysis Bot mentioned this pull request Oct 6, 2026
Closed
hf-security-analysis Bot added a commit that referenced this pull request Oct 6, 2026
@hf-security-analysis
fix(ci): harden workflow files flagged on #49357
a6cd761
@hf-security-analysis hf-security-analysis Bot mentioned this pull request Oct 6, 2026
Closed
@HuggingFaceDocBuilderDev

HuggingFaceDocBuilderDev commented Oct 6, 2026

Copy link
Copy Markdown

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@SunMarc
SunMarc requested a review from vasqu October 6, 2026 15:33
@vasqu

vasqu commented Oct 7, 2026

Copy link
Copy Markdown
Collaborator

cc @huggingface/transformers-core-maintainers not sure if im the right one to judge here but it's based on https://huggingface.slack.com/archives/C01N44FJDHT/p1791296842367149?thread_ts=1791268994.125769&cid=C01N44FJDHT

@ArthurZucker

ArthurZucker commented Oct 8, 2026

Copy link
Copy Markdown
Collaborator

YEP makes sense. @3outeille for the 3d parallel? tho might be outdated?

@SunMarc
SunMarc requested a review from 3outeille October 8, 2026 13:01
hf-security-analysis Bot added a commit that referenced this pull request Oct 8, 2026
@hf-security-analysis
fix(ci): harden workflow files flagged on #49357
f9a2c39
@hf-security-analysis hf-security-analysis Bot mentioned this pull request Oct 8, 2026
Closed
@3outeille

3outeille commented Oct 9, 2026

Copy link
Copy Markdown
Member

3D parallel example-> do we use those no to stress test our 3D integration ? Happy to keep them otherwise

No we dont use those to stress test the 3D integration. Can be deleted

vasqu
vasqu approved these changes Oct 12, 2026
View reviewed changes

@vasqu vasqu left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's get them removed 🫡

hf-security-analysis Bot added a commit that referenced this pull request Oct 12, 2026
@hf-security-analysis
fix(ci): harden workflow files flagged on #49357
effdeac
@hf-security-analysis hf-security-analysis Bot mentioned this pull request Oct 12, 2026
Closed
hf-security-analysis Bot added a commit that referenced this pull request Oct 12, 2026
@hf-security-analysis
fix(ci): harden workflow files flagged on #49357
08e9510
@hf-security-analysis hf-security-analysis Bot mentioned this pull request Oct 12, 2026
Closed
hf-security-analysis Bot added a commit that referenced this pull request Oct 13, 2026
@hf-security-analysis
fix(ci): harden workflow design flagged on #49357
d4d30e2
@hf-security-analysis hf-security-analysis Bot mentioned this pull request Oct 13, 2026
Closed
hf-security-analysis Bot added a commit that referenced this pull request Oct 13, 2026
@hf-security-analysis
fix(ci): harden workflow design flagged on #49357
ce0cef8
@hf-security-analysis hf-security-analysis Bot mentioned this pull request Oct 13, 2026
Closed
This was referenced Oct 20, 2026
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
@stevhliu stevhliu mentioned this pull request Oct 23, 2026
Closed
This was referenced Oct 30, 2026
Merged
Closed
Merged
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@3outeille 3outeille 3outeille approved these changes
@vasqu vasqu vasqu approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@SunMarc @HuggingFaceDocBuilderDev @vasqu @ArthurZucker @3outeille